Skip to content

fix: outline the selected row only on keyboard focus - #150

Merged
xsmyile merged 1 commit into
masterfrom
xsmyile/row-focus-ring
Oct 7, 2026
Merged

xsmyile merged 1 commit into
masterfrom
xsmyile/row-focus-ring

Conversation

@xsmyile

@xsmyile xsmyile commented Oct 7, 2026

Copy link
Copy Markdown
Member

Replaces the orange left bar, shown on every click, with a rounded outline that appears only on keyboard focus.

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ No new issues found.

Reviewed changes Reviewed the selected-row focus styling and its browser regression coverage.

  • Keyboard-only outline: Replaces the always-focused left marker with a rounded inset outline gated by the listbox’s :focus-visible, preserving the selection background and existing container fallback.
  • Interaction coverage: Updates outline assertions and verifies that pointer clicks suppress the outline while subsequent keyboard navigation restores it.

Validation: all 34 Chromium keyboard-navigation tests passed. The test server also completed TypeScript checking and the production build. Other browsers were not tested.

Pullfrog  | View workflow run | Using gpt-6.1-sol | 𝕏

@xsmyile
xsmyile merged commit 3fbd0e7 into master Oct 7, 2026
3 checks passed
@xsmyile
xsmyile deleted the xsmyile/row-focus-ring branch October 7, 2026 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant